Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement CALLERROR: SecurityError response #354

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

robert-s-ubi
Copy link
Contributor

OCPP 2.0.1 requires a CALLERROR: SecurityError response from the CSMS if the Charging Station sends a request before having been accepted.

Implement a mechanism to send this response, by throwing the newly added SecurityErrorException in the message handler of a disallowed request.

This fixes issue #350.

These commits were based on a misunderstanding of the OCPP-J 1.6
specification, which clearly states that the password is a byte sequence
and not a string:

Revert "More password fixes."

This reverts commit fb5d0fb.

Revert "Fix password decoding."

This reverts commit cf20205.

Revert "Recommended by 1.6 spec is a 20 byte (40 chars) key."

This reverts commit f7b92a3.

This commit breaks the API, because the behaviour of the method is
changed to return the last configuration instead of the default
configuration:

Revert "A single instace, otherwise a static get() method makes no sense."

This reverts commit 953f50b.
OCPP 2.0.1 requires a CALLERROR: SecurityError response from the CSMS
if the Charging Station sends a request before having been accepted.

Implement a mechanism to send this response, by throwing the newly
added SecurityErrorException in the message handler of a disallowed
request.

This fixes issue ChargeTimeEU#350.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant